Add Groundlight.me() for authenticated identity - #468
Open
timmarkhuff wants to merge 13 commits into
Open
Conversation
Expose id, email, username, and customer groups from GET /v1/me so callers can inspect the token's tenant without parsing the raw response. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Avoid colliding with detector-group concepts; callers that need the type can import it from groundlight.identity. Co-authored-by: Cursor <cursoragent@cursor.com>
tomfaulhaber
approved these changes
Jul 31, 2026
| name: str = Field(..., description="Name of the customer group.") | ||
|
|
||
|
|
||
| class Me(BaseModel): # pylint: disable=too-few-public-methods |
Collaborator
There was a problem hiding this comment.
We should be able to get this model for free from the api spec if we model it appropriately in the BE
Contributor
Author
There was a problem hiding this comment.
Done. I opened a separate PR to update the backend.
Replace hand-written identity models with codegen from the updated /v1/me schema, and sync spec/public-api.yaml from zuuul. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Import generated models from model, matching other unit tests. Co-authored-by: Cursor <cursoragent@cursor.com>
Revert accidental full-client regenerate noise and keep only the /v1/me schema and generated Me/CustomerGroup wiring. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Align with zuuul public contract: nested groups use Group, Me omits user id, keep plural groups with id+name. Co-authored-by: Cursor <cursoragent@cursor.com>
Keep /v1/me contract changes in the YAML and model.py Me/Group types, and drop generated openapi-client Me/Group churn. InlineResponse2002 already carries the extra fields through to_dict(). Co-authored-by: Cursor <cursoragent@cursor.com>
Match the public API contract that returns one group object instead of a list. Co-authored-by: Cursor <cursoragent@cursor.com>
Keep parity with the public /v1/me contract that released clients still require. Co-authored-by: Cursor <cursoragent@cursor.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR lets SDK users fetch authenticated identity, including the user's group:
Summary of Changes
Groundlight.me()returning pydanticMe(email,username,group) fromGET /v1/me.Me/Groupmodels ingenerated/model.pyand a surgicalspec/public-api.yamlsync.InlineResponse2002already passes through extra fields viato_dict().whoami()now returnsme().email; CLI picks upmeunder Account.Merge order
group).Test plan
me()/ group models/v1/me)